-
Notifications
You must be signed in to change notification settings - Fork 5.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[cherry-pick] release/2.3 elementwise_mul and matmul mkldnn fix #43725
[cherry-pick] release/2.3 elementwise_mul and matmul mkldnn fix #43725
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
…s outputs (PaddlePaddle#43297) * fix some matmul that X and Y both other ops outputs, do not dequantize the Y. * fix CI format * fix according to review
c07e3bb
to
ce2b2a3
Compare
@jiangjiajun @yeliang2258 Could you please help with the failed CI? I can not see what it is the problem. |
@wozna Please review this PR. Thanks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@yeliang2258 @jiangjiajun @yaomichael The required CI all passed, please review and merge this cherry-pick PR. Thank you! Another cherry-pick PR is #43750, still passing CIs, I have committed with pre-commit. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PR types
Bug fixes
PR changes
OPs
Describe
elementwise_mul and matmul mkldnn fix